usbip: network: Fix unaligned member access
authorBen Hutchings <ben@decadent.org.uk>
Sun, 18 Aug 2019 15:15:26 +0000 (16:15 +0100)
committerSalvatore Bonaccorso <carnil@debian.org>
Sun, 5 Jan 2020 14:40:37 +0000 (14:40 +0000)
commit222c4799bb35403fb57d2240ca1412ba80eccf21
treed0f4a20e3fa59955dd906d87abc5da6ef2e25b13
parent849d2b7efb182a8d5068e4be65b4ad169ad9cf02
usbip: network: Fix unaligned member access

gcc 9 warns:

usbip_network.c: In function ‘usbip_net_pack_usb_device’:
usbip_network.c:79:32: error: taking address of packed member of ‘struct usbip_usb_device’ may result in an unaligned pointer value [-Werror=address-of-packed-member]
   79 |  usbip_net_pack_uint32_t(pack, &udev->busnum);
      |                                ^~~~~~~~~~~~~

and similarly for other calls to usbip_net_pack_uint{16,32}_t().

These fields are unaligned because they are declared as part of a
packed structure.  Functions operating on the structure will use the
appropriate accessors for unaligned data if necessary, but there
doesn't seem to be a way to declare functions as taking a pointer
to a unaligned scalar.

Instead, change these functions to take a pointer of type void * and
to memcpy() the unaligned value in and out of a local variable.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Gbp-Pq: Topic bugfix/all
Gbp-Pq: Name usbip-network-fix-unaligned-member-access.patch
tools/usb/usbip/src/usbip_network.c
tools/usb/usbip/src/usbip_network.h